Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: change mount correctly in protection zone #2239

Merged
merged 17 commits into from
Mar 5, 2024
Merged

Conversation

dudantas
Copy link
Member

@dudantas dudantas commented Feb 15, 2024

Prevents "cloning" of speed using mount when it is enabled to be used in pz.

@dudantas dudantas marked this pull request as draft February 15, 2024 22:06
@YagoOliveiraB
Copy link

Olá, boa tarde!
Aqui essa solução não deu certo não !

2024-02-17.12-33-44.mp4

@dudantas dudantas marked this pull request as ready for review February 18, 2024 20:44
@dudantas
Copy link
Member Author

Olá, boa tarde! Aqui essa solução não deu certo não !

2024-02-17.12-33-44.mp4

Opa, testei aqui e funcionou.

@elsongabriel
Copy link
Contributor

elsongabriel commented Feb 20, 2024

Olá, boa tarde! Aqui essa solução não deu certo não !
2024-02-17.12-33-44.mp4

Opa, testei aqui e funcionou.

edu, eu testei aqui, e realmente não está clonando.
mas encontrei alguns problemas:

  • quando eu deslogo sem a montaria, ao logar ele coloca a montaria e o bônus (achei de boa isso).
  • já logado com a situação anterior, ao retirar a montaria e adicionar novamente, o bônus não é adicionado.
  • se eu retirar a montaria, o bônus é removido, ficando com a speed menor que o normal.
  • em outro teste, eu usei o mount (do client), depois tirei a montaria no customize character, e usei novamente o mount e "clonou" mais 10 no speed.

video de teste:
mount_test
teste dentro de PZ:
mount_test2

obs: estou na sua branch atualizada.
pelo "mount" e "desmount" do client, funciona normal.

@elsongabriel elsongabriel self-requested a review February 20, 2024 14:21
@YagoOliveiraB
Copy link

YagoOliveiraB commented Feb 20, 2024

@elsongabriel .

No config.lua se você ativar/desativar a opção toggleMountInProtectionZone tem ações diferentes.
toggleMountInProtectionZone = false
Você não consegue clonar o speed.

toggleMountInProtectionZone = true
Speed infinito, parece que ele ignora.

conversei com o @dudantas no discord e o mesmo confirmou minha afirmação, porém parece que ficou sem tempo pra "rever".

@elsongabriel
Copy link
Contributor

elsongabriel commented Feb 20, 2024

@elsongabriel Está testando errado.

No config.lua se você ativar/desativar a opção toggleMountInProtectionZone tem ações diferentes. toggleMountInProtectionZone = false Você não consegue clonar o speed.

toggleMountInProtectionZone = true Speed infinito, parece que ele ignora.

conversei com o @dudantas no discord e o mesmo confirmou minha afirmação, porém parece que ficou sem tempo pra "rever".

então cara, não testei errado.. eu testei das 2 formas (toggleMountInProtectionZone true e false).
fiz a alteração, e testei novamente das 2 formas e dei a sugestão da correção.

@elsongabriel elsongabriel self-requested a review February 20, 2024 18:51
@dudantas dudantas marked this pull request as draft February 23, 2024 20:46
@elsongabriel elsongabriel mentioned this pull request Feb 24, 2024
5 tasks
@dudantas dudantas mentioned this pull request Feb 24, 2024
@dudantas dudantas marked this pull request as ready for review February 24, 2024 22:20
otmexa pushed a commit to bennyhdz/canaryfork that referenced this pull request Feb 27, 2024
@elsongabriel elsongabriel marked this pull request as draft February 29, 2024 14:20
@dudantas dudantas changed the title fix: mount speed bug fix: change mount correctly in protection zone Mar 5, 2024
@dudantas dudantas marked this pull request as ready for review March 5, 2024 12:36
@omarcopires
Copy link
Contributor

works fine!

@dudantas dudantas merged commit 9a4b49d into main Mar 5, 2024
32 checks passed
@dudantas dudantas deleted the dudantas/fix-mount-bugs branch March 5, 2024 21:00
odisk777 added a commit to odisk777/canary that referenced this pull request Mar 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants